Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/room booking #245

Open
wants to merge 33 commits into
base: main
Choose a base branch
from

Conversation

BrandonDoza
Copy link

Pull Request

Description

This PR allows a user to book a room, and cancel a booking

Related Issues

Fixes the issue of a user not being able to book a room or cancel a booking

Changes Made

With the feature addition, a user can now book a room, view that room in their booked rooms view, and from there they can choose to cancel a booking if they wish. They can book rooms by date, and filter those by room type as well.

Testing

Testing was done locally to make sure the expected outcomes we populating on the DOM. Since we are not testing DOM functions directly, the application was gone over with different paths the user may take to assure the outcome is as expected.

Checklist

  • [ x ] The code follows the project's coding standards.
  • Unit tests have been added or updated to cover the changes.
  • Documentation has been updated to reflect the changes (if applicable).
  • [ x ] The code compiles without errors.
  • [ x ] The changes have been tested locally and pass all relevant tests.
  • [ x ] All new and existing tests pass.
  • The pull request has been reviewed by at least one other contributor.

Reviewer Instructions

When reviewing please refer to the domUpdates.js file to view the changes that were made.

BrandonDoza and others added 30 commits April 16, 2024 19:04
Write User Tests and Functions to Solve
Feature/view user bookings - completion of iteration 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants